Skip to content

Conversation

@vojtechtrefny
Copy link
Member

@vojtechtrefny vojtechtrefny commented Apr 3, 2025

nvme_get_sanitize_log and nvme_get_self_test_log now return some invalid data for our test loop based NVMe devices. We'll skip the tests until this is resolved.

See also #1098 and linux-nvme/libnvme#980

nvme_get_sanitize_log and nvme_get_self_test_log now return some
invalid data for our test loop based NVMe devices. We'll skip the
tests until this is resolved.
Copy link
Member

@tbzatek tbzatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, just don't forget to revert this once we have proper libnvme fix!

@vojtechtrefny
Copy link
Member Author

Closing, the fix will be released soon in Fedora and it doesn't make sense to merge this for just few days and then revert it again.

@tbzatek
Copy link
Member

tbzatek commented Apr 3, 2025

@vojtechtrefny
Copy link
Member Author

Seems to fix only half of the problem. test_self_test_log now works as expected, but test_sanitize_log now fails but with a different error message:

AssertionError: "NVMe Get Log Page - Sanitize Status Log command error: Invalid Field in Command: A reserved coded value or an unsupported value in a defined field|NVMe Get Log Page - Sanitize Status Log command error: unrecognized" does not match "g-bd-nvme-error-quark: NVMe Get Log Page - Sanitize Status Log command error: No such file or directory (1)"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants